test with, and support, auth framework specifically.#776
Draft
d-w-moore wants to merge 2 commits intoirods:mainfrom
Draft
test with, and support, auth framework specifically.#776d-w-moore wants to merge 2 commits intoirods:mainfrom
d-w-moore wants to merge 2 commits intoirods:mainfrom
Conversation
Collaborator
Author
|
@d-w-moore Reminder - rebase to |
8513424 to
580f23a
Compare
…eation]
[_697] Github workflows for running PRC tests
Co-authored-by: Terrell Russell <tgr@renci.org>
Includes a docker compose configuration in which the PRC test suite
can be run in a way representative of typical operational use, and the
version of iRODS server and python interpreter that we install are also easily
reconfigurable.
Additionally, this affords us the opportunity to run the test suite on a client
node all its own, the iRODS server being reachable only via the network.
[_502] test harness and container-based tests
A new test harness is introduced in which we construct a new
container (using either Docker or podman) for each test program
we run. This allows full customization of the container
environment for the particular needs of each test.
Accordingly, included with the Github workflows (in a separate
commit) is a full run of the PRC test suite with the iRODS server
and catalog DB server running in the same container as the client.
In the process of putting old tests through new rigors, faults
were found and corrected in some of those tests.
completely remove {send,recv}_oneshot sync mechanism
review comments
Comments moved to *.yml files. README.md no longer needed.
update irods/harness/README
deindent
correct forgotten comment symbol
clearer comment in run_tests.sh
suggestions nr. 1 10-22-25 13:19
suggestions nr. 2 10-22-25
smchg
package repo chg
comments chg
prc_write_irodsA.py invokes a function in the irods.client_init module which adapts to iRODS server version, using the new authentication framework in iRODS 4.3+ and legacy auth in previous versions.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR will likely include tests in near future that specifically target the
pam_passwordandpam_interactiveschemes.We want to